Upgrade to polkadot 1.1.0 #1004
Conversation
| orml-vesting = { default-features = false, git = "https://github.com/open-web3-stack/open-runtime-module-library", branch = "polkadot-v0.9.43" } | ||
| orml-xcm-support = { default-features = false, git = "https://github.com/open-web3-stack/open-runtime-module-library", branch = "polkadot-v0.9.43" } | ||
| orml-xtokens = { default-features = false, git = "https://github.com/open-web3-stack/open-runtime-module-library", branch = "polkadot-v0.9.43" } | ||
| orml-tokens = { default-features = false, git = "https://github.com/moonbeam-foundation/open-runtime-module-library", branch = "upgrade-to-polkadot-v1.1.0" } |
There was a problem hiding this comment.
Нам норм от Moonbeam зависеть?
There was a problem hiding this comment.
PR ещё не принят, но там всё в целом ок.
| fp-rpc = { workspace = true } | ||
|
|
||
| app-promotion-rpc = { workspace = true } | ||
| fc-api.workspace = true |
There was a problem hiding this comment.
Как-то не консистентно. В одних местах через { workspace = true }, а в других .workspace = true
There was a problem hiding this comment.
Какие-то вещи руками изменялись, какие-то cargo upgrade/cargo add трогал, какие-то через deppatcher патчились...
There was a problem hiding this comment.
По идее такое taplo должен уметь, но я не вижу у него опции этой.
Могу в deppatcher написать форматирование такого, по идее не сложно)
There was a problem hiding this comment.
Сам я считаю что лучше = {workspace = true} делать, поскольку так проще новые поля добавлять
| #[cfg_attr(feature = "serde1", derive(Serialize, Deserialize))] | ||
| #[derive( | ||
| Encode, Decode, Debug, Clone, PartialEq, TypeInfo, MaxEncodedLen, Serialize, Deserialize, | ||
| )] | ||
| pub enum MetaUpdatePermission { |
There was a problem hiding this comment.
Судя по всему, мы могли бы это убрать, все миграции релеватные давно прошли
|
@CertainLach в основном только вопросы и пара nitpicks |
Co-authored-by: Daniel Shiposha <[email protected]>
|
Нода будет отдельным PRом |
No description provided.